Conversation
irisqul
left a comment
irisqul
left a comment
There was a problem hiding this comment.
✅ Сильные стороны:
-
Код запускается без ошибок
-
Обеспечено необходимое тестовое покрытие
| 'Плоды Фалленианского дерева', | ||
| 'Кристаллы марсианских альфа-сахаридов', | ||
| 'Мини-салат Экзо-Плантаго', | ||
| 'Сыр с астероидной плесенью' |
There was a problem hiding this comment.
Можно лучше: для параметризации используют данные которые отличаются друг от друга, например принадлежат к разным классам эквивалентости (либо тестируют границы одного класса) и имеют одинаковый ОР. Твой список - это по сути один и тот же класс, нам не нужно проверять его много раз на самом деле.
| ingredient.name = ingredient_name | ||
| burger = Burger() | ||
| burger.add_ingredient(ingredient) | ||
| assert len(burger.ingredients) == 1 |
There was a problem hiding this comment.
Нужно исправить: тест неатомарный. после ассерта уже не должно быть шагов, а каждый тест должен проверять какое то одно поведение приложения
| burger.add_ingredient(ingredient1) | ||
| burger.add_ingredient(ingredient2) | ||
| assert burger.ingredients[0].name == ingredient1_name | ||
| assert burger.ingredients[1].name == ingredient2_name |
There was a problem hiding this comment.
Нужно исправить: тест неатомарный. после ассерта уже не должно быть шагов, а каждый тест должен проверять какое то одно поведение приложения
| ('Соус Spicy-X', 'Мясо бессмертных моллюсков Protostomia'), | ||
| ('Соус фирменный Space Sauce', 'Говяжий метеорит (отбивная)'), | ||
| ('Соус традиционный галактический', 'Биокотлета из марсианской Магнолии'), | ||
| ('Соус с шипами Антарианского плоскоходца', 'Хрустящие минеральные кольца') |
There was a problem hiding this comment.
правда ли нужно проверять все эти комбинации? Узнаем ли мы что-то новое повторяя этот тест?
2 participants
Разработаны юнит-тесты для программы, которая помогает заказать бургер в Stellar Burgers